Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement text wrapping for most diagnostic messages #446

Merged
merged 5 commits into from
Feb 12, 2025
Merged

Conversation

mcy
Copy link
Member

@mcy mcy commented Feb 11, 2025

This PR adds basic text wrapping to diagnostic messages (except those attached to snippets) to avoid needing to wrap notes/helps by hand when writing diagnostics.

@mcy mcy requested a review from doriable February 11, 2025 23:09
@mcy mcy requested a review from doriable February 11, 2025 23:25
@mcy mcy enabled auto-merge (squash) February 11, 2025 23:25
@mcy mcy disabled auto-merge February 12, 2025 22:54
@mcy mcy merged commit 797d668 into main Feb 12, 2025
8 checks passed
@mcy mcy deleted the mcy/report-wrapping branch February 12, 2025 22:59
mcy added a commit that referenced this pull request Feb 12, 2025
This PR adds basic text wrapping to diagnostic messages (except those
attached to snippets) to avoid needing to wrap notes/helps by hand when
writing diagnostics.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants